Skip to content

[codex] refactor(config): extract harness posture module#3506

Closed
Hmbown wants to merge 1 commit into
mainfrom
codex/config-extract-harness-posture
Closed

[codex] refactor(config): extract harness posture module#3506
Hmbown wants to merge 1 commit into
mainfrom
codex/config-extract-harness-posture

Conversation

@Hmbown

@Hmbown Hmbown commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Moves the harness posture/profile data model out of crates/config/src/lib.rs into a focused crates/config/src/harness.rs module.
  • Moves the built-in harness profile seeds and profile matching helpers with the type they support, keeping the boundary cohesive.
  • Re-exports the harness types and built_in_harness_profiles from the crate root so existing callers keep the same public API.

Refs #3311

Testing

  • cargo fmt --all -- --check
  • cargo clippy -p codewhale-config --all-targets --locked
  • cargo test -p codewhale-config --locked harness
  • cargo test -p codewhale-config --locked config
  • cargo test -p codewhale-config --locked
  • git diff --check
  • cargo clippy --workspace --all-targets --all-features
  • cargo test --workspace --all-features

Checklist

  • Updated docs or comments as needed (moved existing rustdoc with the types)
  • Added or updated tests where relevant (existing harness/config coverage preserved)
  • Verified TUI behavior manually if UI changes (N/A, config crate extraction only)
  • Harvested/co-authored credit uses a GitHub numeric noreply address (N/A, no harvested code)

Move the harness posture/profile data model, built-in seeds, and profile matching helpers out of lib.rs into a focused harness module.\n\nRe-export the public harness types and built_in_harness_profiles from the crate root so existing callers and tests keep the same API.\n\nThis is a behavior-preserving config split slice for #3311.
@github-actions

Copy link
Copy Markdown

Thanks @Hmbown — your contribution landed in 5f07344826d8 on main:

refactor(config): finish harness posture split

Closing this PR now that the code is on main. Credit lives in the commit message and (where applicable) the CHANGELOG.md entry for the next release. Apologies for not closing this at the time of the merge — the auto-close workflow is new in v0.8.31.

If you want to land more work and would prefer your future PRs merge cleanly without a harvest step, the CONTRIBUTING.md doc has a short note on what makes a contribution mergeable as-is.

@github-actions github-actions Bot closed this Jun 24, 2026
@Hmbown

Hmbown commented Jun 24, 2026

Copy link
Copy Markdown
Owner Author

Closed as superseded by #3560 (merge ec29998), which landed the remaining harness-profile split on current main.

pull Bot pushed a commit to indielab/CodeWhale that referenced this pull request Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant